-
-
Notifications
You must be signed in to change notification settings - Fork 674
feat: add support for ravencoin #1627
base: dev
Are you sure you want to change the base?
Conversation
stuff wip fix: native bindings wip fuck yeah wip payout fix payout cleanup raven -> ravencoin more cleanup
Nice :) |
I would like to try raven coin. Is this merged into miningcore or do I need to submit a request from jon4hz? |
no, this isn't merged yet. Hence the PR.. |
Cool. Hopefully oliverw will merge the pr soon. If I clone your repo will
it work? If I checkout the branch ravencoin?
…On Wed, Feb 22, 2023, 7:05 PM jon4hz ***@***.***> wrote:
no, this isn't merged yet. Hence the PR..
—
Reply to this email directly, view it on GitHub
<#1627 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIAKRQHBKXZEEFDLTXHZK3WY2SURANCNFSM6AAAAAAU4DJW24>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Probably, yes |
Tested this project conditionally working there are many inaccuracies such as diff target
and OnSubscribeAsync
I didn’t check the block on the test network, I didn’t catch the payout yet, but I’m going to do it and write it. |
Thanks a lot for taking a look at this! Good hint with the target encoding. I was actually quite unsure if I handled that correctly but I kinda forgot about it since the code seemed to work. (It actually does work but I think I'm loosing some precision down the line.) I ported the share validation from blinkhash's foundation pool and I never heard that they do it wrong. What makes you think that Diff1 should be As for the subscribe params: wildrig accepts both variants. Is there a particular reason to only send null + extraNonce1? |
No it does not. |
Stand corrected, not sure if its because of old base for me, but I also needed to add:
to clusterconfigextension so it would not use the headerhasher as displayed algo. |
Good catch! Fixed in 1f3df14 |
https://github.com/nicehash/Specifications/blob/master/KawPow_NiceHash_v1.0.txt |
When testing the same problem as Ethash: Abstract and light cache jumping pings on Ethash tested with dag generation there is a stable ping difference with my pool of 1ms it turns out that with dag is much better. If you use light cache then you also need to generate cache files and test them. |
I'm no expert on ethash by all means but I think the performance difference can be explained, because the light cache has to calculate the required dag sequences on the fly, where as the dag has them already precalculated. See: https://ethereum.org/en/developers/docs/consensus-mechanisms/pow/mining-algorithms/ethash/
How big was the performance difference in your experience? If it's only a ms I dare to say the advantage of being able to scale overwhelms the disadvantages. But yeah, that all depends on how big the difference is.
I though that was already handled by the c++ code but I'll take another look. Doesn't it store the light cache in-memory? |
Hello. I try kawpow algo on hivecoin. Have error with block submission: submission failed with: bad-cb-community-autonomous-amount |
You need to add support for "CommunityAutonomousAddress" and "CommunityAutonomousValue" as those are different to founder/payee/masternode/minerfund. Check the minerfund commit and follow that. Pretty easy to add. I did lazy way and just added to getblocktemplate. Was going to do it properly with its own file and submit a pull request to jon4hz, but got distracted
in getblocktemplate
in clusterconfig
I think didn't miss anything, but might have. |
I add all this code, recompile. But submission failed again. |
did you also add "hasCommunityAddress": true in to the coins.json for the coin? |
No. just add code. Thank you, now try add to coins.json |
After add "hasCommunityAddress": true block submission done! Thank you. |
Find new issue. Payments work, but there are no records in the database about payments made. No records in "payments" in db. |
Are you using the same wallet for the pool fee and mining? |
Yes 1 wallet for fee and mining. Now try change wallet |
Miningcore doesn't store payments sent to the fee wallet. You'll have to use a different wallet for mining if you want to test payouts. |
Thank you. Understand. Now change wallet |
Have new problem :) |
Getting OT on kawpow, but paprikacoin has incorrect coinbasevalue on getblocktemplate, I submitted a pull request to fix it. In the meanwhile, check the pull request in paprikacoin git, apply it and build from source. |
@papagruz This PR is about ravencoin and not some random other coin. Please keep this thread on topic and open another issue if you have issues with paprikacoin... |
Ok. Thanks |
Read the comments! This PR is for Ravencoin. Not other coins based on kawpow. Open a new discussion for the coin you are trying to work with |
Sorry i'm a little bit late to the party. I tried to mine RVN
Command lines for both miners:
Am i the only one having that issue? |
@blackmennewstyle that isn't an issue, just the miner reporting its hashrate to the pool. Miningcore doesn't support that though, so the message gets ignored. |
Hi, sorry for the confusion i wanted to write behavior not issue, another brain glitch i guess. Everything is working fine, i had to decrease my minDiff though because my GPU for testing isn't really great at KAWPOW lol Nice job mate 👍 |
Did anyone manage to successfully catch a block on Ravencoin |
Yes. How long have you been mining and what is your hashrate on the pool?
On Mon, Jun 26, 2023 at 1:42 AM Cédric CRISPIN ***@***.***> wrote:
Have anyone managed to successfully catch a block on Ravencoin mainnet?
—
Reply to this email directly, view it on GitHub
<#1627 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIAKRUSN4WWHD2O4EK5GLTXNEONRANCNFSM6AAAAAAU4DJW24>
.
You are receiving this because you commented.Message ID:
***@***.***>
--
Randy Hasson
199 East Markison Avenue
Columbus, Ohio 43207
|
Almost 4 days with just 130 MH/s lol Did you implement @Konstantin35 suggestions with the target encoding and also the shareDiff calculation? |
With 130 mh/s it will take weeks if not months to find a block
On Mon, Jun 26, 2023 at 10:26 AM Cédric CRISPIN ***@***.***>
wrote:
Yes. How long have you been mining and what is your hashrate on the pool?
On Mon, Jun 26, 2023 at 1:42 AM Cédric CRISPIN *@*.*> wrote: Have anyone
managed to successfully catch a block on Ravencoin mainnet? — Reply to this
email directly, view it on GitHub <#1627 (comment)
<#1627 (comment)>>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIAKRUSN4WWHD2O4EK5GLTXNEONRANCNFSM6AAAAAAU4DJW24
<https://github.com/notifications/unsubscribe-auth/AAIAKRUSN4WWHD2O4EK5GLTXNEONRANCNFSM6AAAAAAU4DJW24>
. You are receiving this because you commented.Message ID: @.*>
-- Randy Hasson 199 East Markison Avenue Columbus, Ohio 43207
<https://www.google.com/maps/search/199+East+Markison+Avenue+Columbus,+Ohio+43207?entry=gmail&source=g>
Almost 4 days with just 130 MH/s lol
Did you implement @Konstantin35 <https://github.com/Konstantin35>
suggestions with the target encoding and also the shareDiff calculation?
—
Reply to this email directly, view it on GitHub
<#1627 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIAKRWKRVYN5RZF3OBLUMTXNGLY5ANCNFSM6AAAAAAU4DJW24>
.
You are receiving this because you commented.Message ID:
***@***.***>
--
Randy Hasson
199 East Markison Avenue
Columbus, Ohio 43207
|
I know lol But i think there is an issue with target encoding and shareDiff calculation because i'm getting from time to time, few low difficulty rejected shares and i think the modifications suggested by @Konstantin35 would fix them. So i imagine you didn't implement them into your pool then since you did not answer lol |
I implemented what is suggested here #1627 (comment) After restart, all my low difficulty rejected shares are gone. That GPUs rig was mining on
I use |
Hi, this PR adds support for ravencoin and lays the basis for other kawpow coins.
The code was tested on the ravencoin testnet.
Even though ravencoin is a fork of bitcoin, it has a different stratum protocol. This is why I added it as a custom family which extends the
BitcoinJob
andBitcoinJobManagerBase
classes.Payouts are handled by the
BitcoinPayoutHandler
without any modifications.rel:
#1174
#876